Auto merge of #2307 - alexcrichton:readme, r=brson

Running `cargo build` should work just fine to build cargo as well as not
running the python script to download rustc itself. Now that Cargo runs on
stable (and a pretty old stable) most Rust installations should "Just Work" to
build Cargo.

Add more info to README about compiling

Running `cargo build` should work just fine to build cargo as well as not
running the python script to download rustc itself. Now that Cargo runs on
stable (and a pretty old stable) most Rust installations should "Just Work" to
build Cargo.

Merge pull request #2303 from alexcrichton/bump

Bump to 0.9.0

Bump to 0.9.0

Auto merge of #2299 - retep998:staticlib-naming-fiasco, r=alexcrichton

Required for https://github.com/rust-lang/rust/pull/29520

r? @alexcrichton

Support static libraries with names other than libfoo.a
Required for https://github.com/rust-lang/rust/pull/29520

Signed-off-by: Peter Atashian <retep998@gmail.com>

Auto merge of #2296 - alexcrichton:wow-thats-a-bad-bug, r=brson

There was a previous bug where if any crate's name was the start of the crate
being updated that the crate wouldn't resolve at all, and this fixes the prefix
checking error bug.

cc #2293

Auto merge of #2270 - felixc:warn-on-empty-dep, r=alexcrichton

This warns when encountering dependencies of either of these forms:
```
  [dependencies]
  foo = {}
```
and
```
  [dependencies.foo]
```
(with nothing further provided).

In the future, this should likely become a hard error.

Related to #2147

Emit a warning when manifest specifies empty dependency constraints

This warns when encountering dependencies of either of these forms:

  [dependencies]
  foo = {}

and

  [dependencies.foo]

(with nothing further provided).

In the future, this should likely become a hard error.

Auto merge of #2282 - mcarton:clippy, r=alexcrichton

For information, here is the [log before and after](https://gist.github.com/mcarton/684c030321c4c60d6bc9) with Clippy.
Remaining warnings from Clippy are mostly false positive or `str` to `string` conversion (too many of them for this PR).

```
warnings before:
     cmp_owned : 2
     cyclomatic_complexity : 1
     deprecated : 11
     explicit_iter_loop : 57
     identity_op : 3
     len_without_is_empty : 3
     len_zero : 20
     let_and_return : 3
     map_clone : 14
     needless_lifetimes : 4
     needless_return : 24
     ok_expect : 2
     option_map_unwrap_or : 10
     private_in_public : 28
     redundant_closure : 2
     should_implement_trait : 2
     single_match : 3
     str_to_string : 112
     string_to_string : 12
     type_complexity : 3
     unnecessary_mut_passed : 2
     unneeded_field_pattern : 3
     unused_lifetimes : 1
     unused_variables : 2
     while_let_loop : 2

warnings after:
     cmp_owned : 2
     cyclomatic_complexity : 1
     identity_op : 1
     let_and_return : 3
     map_clone : 1
     needless_return : 3
     ok_expect : 2
     option_map_unwrap_or : 1
     private_in_public : 28
     should_implement_trait : 2
     str_to_string : 80
     type_complexity : 3
     while_let_loop : 2
```

Fix update --precise with the registry source

There was a previous bug where if any crate's name was the start of the crate
being updated that the crate wouldn't resolve at all, and this fixes the prefix
checking error bug.

cc #2293

Auto merge of #2285 - sfackler:revert-wildcard-warning, r=alexcrichton

The crates.io change is going through, so we don't want this warning
around after it lands.

r? @alexcrichton

This would perferrably make it into the Cargo that's shipping with 1.6.

Auto merge of #2287 - alexcrichton:openbsd, r=alexcrichton

cc @semarie

Register an OpenBSD snapshot

Drop wildcard crates.io warning

The crates.io change is going through, so we don't want this warning
around after it lands.

Use .cloned() where appropriate

Fix most of Clippy’s map_clone warnings.

Remove some useless String::to_string calls

Fix some Clippy’s string_to_string warnings.

Remove unnecessary mut

Fix all of Clippy’s unnecessary_mut_passed warnings.

Replace single matches by ifs

Fix all Clippy’s single_match warnings.

Remove redundant closure

Fix all Clippy’s redundant_closure warnings.

Use .is_empty() where appropriate

Fix all of Clippy’s len_zero warnings.

Add an is_empty method to types with a len one

Fix all Clippy’s len_without_is_empty warnings.

Remove useless returns

Fix most of Clippy’s needless_return warnings. Remaining cases are
false positives.

Remove useless lifetimes

Fix all Clippy’s needless_lifetimes warnings.

Auto merge of #2281 - rillian:doc-nocapture, r=alexcrichton

Although this is a feature of rustc's test harness and
not cargo's test driver, I can never remember how to
invoke it. Describing the switch in `cargo test --help`
makes it more discoverable.

Document --nocapture to reduce confusion running tests.

Although this is a feature of rustc's test harness and
not cargo's test driver, I can never remember how to
invoke it. Describing the switch in `cargo test --help`
makes it more discoverable.

Auto merge of #2279 - alexcrichton:rerun-if-changed-rust-file, r=brson

There was a failure mode of the handling of the rerun-if-changed directive where
it would rerun the build script twice before hitting a steady state of actually
processing the directives. The order of events that led to this were:

1. A project was built from a clean directory. Cargo recorded a fingerprint
   indicating this (for the build script), but the fingerprint indicated that
   the build script was a normal build script (no manually specified inputs)
   because Cargo had no prior knowledge.
2. A project was then rebuilt from the same directory. Cargo's new fingerprint
   for the build script now indicates that there is a custom list of
   dependencies, but the previous fingerprint indicates there wasn't, so the
   mismatch causes another rebuild.
3. All future rebuilds will agree that there are custom lists both before and
   after, so the directives are processed as one would expect.

This commit does a bit of refactoring in the fingerprint module to fix this
situation. The recorded fingerprint in step (1) is now recorded as a "custom
dependencies are specified" fingerprint if, after the build script is run,
custom dependencies were specified.

Closes #2267

Handle rerun-if-changed directives in dependencies

There was a failure mode of the handling of the rerun-if-changed directive where
it would rerun the build script twice before hitting a steady state of actually
processing the directives. The order of events that led to this were:

1. A project was built from a clean directory. Cargo recorded a fingerprint
   indicating this (for the build script), but the fingerprint indicated that
   the build script was a normal build script (no manually specified inputs)
   because Cargo had no prior knowledge.
2. A project was then rebuilt from the same directory. Cargo's new fingerprint
   for the build script now indicates that there is a custom list of
   dependencies, but the previous fingerprint indicates there wasn't, so the
   mismatch causes another rebuild.
3. All future rebuilds will agree that there are custom lists both before and
   after, so the directives are processed as one would expect.

This commit does a bit of refactoring in the fingerprint module to fix this
situation. The recorded fingerprint in step (1) is now recorded as a "custom
dependencies are specified" fingerprint if, after the build script is run,
custom dependencies were specified.

Closes #2267

Auto merge of #2276 - alexcrichton:fix-nightly, r=alexcrichton

Just some tweaks to compiler error messages

Auto merge of #2228 - tbu-:pr_only_missing, r=alexcrichton

Fix tests on nightly

Auto merge of #2269 - felixc:package-untracked-files, r=alexcrichton

The logic in `sources/path.rs` that finds files that are untracked by the version control system was incorrectly including files that *are* tracked, but had been modified without the modification being committed to Git.

This manifested itself as strange duplications of files (e.g. in `cargo package --list`), since they were listed once as tracked files, and once more as allegedly "untracked" ones. It also caused failures when trying to package the crate when files had been deleted, but the deletion was not yet committed to Git.

Fixes #2199

Fix listing of untracked files to not include tracked but modified ones

The logic in `sources/path.rs` that finds files that are untracked by the
version control system was incorrectly including files that *are* tracked,
but had been modified without the modification being committed to Git.

This manifested itself as strange duplications of files (e.g. in `cargo
package --list`), since they were listed once as tracked files, and once
more as allegedly "untracked" ones. It also caused failures when trying to
package the crate when files had been deleted, but the deletion was not
yet committed to Git.

Fixes #2199

Bump rand to 0.3.13

Auto merge of #2265 - dirk:dirk/include-publish-command-in-help, r=alexcrichton

This is pretty commonly-used command, so I think it merits inclusion in the help text.

Auto merge of #2257 - nrc:lib-template, r=alexcrichton

Only treat missing crate lists as empty

The motivation is that errors other than "file not found" could lead to
the file being overwritten by an empty config, e.g. if the `open` call
is interrupted and returns `EINTR`, then this code would read an empty
config and write it back later on. This can probably happen for other
errors as well (`ENFILE`, ...).

Add publish command to list of commands in help text

Merge pull request #2260 from durka/patch-4

doc/manifest: add `cargo run --example`

Merge pull request #2243 from felixc/zsh-completion-options

Complete the definition of all options for zsh auto-completion

doc/manifest: add `cargo run --example`

Closes #2251. Should it be mentioned anywhere else as well?

Tweak the new project template to be more idiomatic.

Complete the definition of all options for zsh auto-completion

All commands can now auto-complete all of the options they support.

Auto merge of #2237 - sfackler:libc-0.2, r=alexcrichton

Also update everything else which removes all remaining libc 0.1
dependencies

Fix windows

Whitelist improper ctypes to support old rustc versions

Bump libc to 0.2

Also update everything else which removes all remaining libc 0.1
dependencies

Auto merge of #2225 - sgrif:sg-doctest-link-args, r=alexcrichton

It is entirely possible for a crate to have a build script that is simply
the equivalent to

```rustc
fn main() {
    println!("cargo:rustc-link-search=native=/some/path");
}
```

Without actually giving anything to link (for example, because the code
contains `#[link(name="foo")]`. In this case, we aren't actually passing
`-L` through when running doctests, even though they're passed when
compiling the main crate.

Fixes #1592

Remove needless dylib check, add test

In this case the dylib check isn't actually doing anything useful, as
we're just appending search paths. Also adds a test for
8c65284b44337c6cfc003cedc8996e241ac678bd

Auto merge of #2233 - gentoo90:bash-comp, r=alexcrichton

Auto merge of #2235 - felixc:zsh-completion, r=alexcrichton

Adds missing rustc, rustdoc, search, and uninstall subcommands.

Removes obsolete config-for-key and config-list subcommands that don't exist anymore.

Add missing subcommands to zsh auto-completion

Specifically, rustc, rustdoc, search, and uninstall.

Remove obsolete commands from zsh auto-completion

The config-for-key and config-list subcommands don't exist.

Bashcomp: complete --bin, --bench and --test

Auto merge of #2232 - alexcrichton:unused, r=huonw

This includes alexcrichton/toml-rs@39d77b6d which ensures that more unused
fields stick around when decoding TOML, allowing this to:

Closes #2230

Auto merge of #2231 - felixc:docstrings-for-all, r=alexcrichton

Almost all commands have info on what they are for, but a few were
missing it.

Also cleans up some copy/paste misdocumentation along the way.

Update toml dependency to catch more unused keys

This includes alexcrichton/toml-rs@39d77b6d which ensures that more unused
fields stick around when decoding TOML, allowing this to:

Closes #2230

Ensure all subcommands have summary description documentation

Almost all commands have info on what they are for, but a few were
missing it.

Also cleans up some copy/paste misdocumentation along the way.

Auto merge of #2226 - tshepang:unused, r=alexcrichton

Auto merge of #2224 - alexcrichton:rustc-dev, r=brson

This should allow compiling the specified target in the various profiles that
are available to it, e.g. bench or test in addition to the standard
dev.

Closes #2120

Auto merge of #2223 - alexcrichton:better-dev-experience, r=brson

Each test wants to be sure to reset HOME and remove CARGO_HOME from the
environment, but this was done inconsistently throughout the test suite. This
commit consolidates process creation so there's only one point for creating a
process ready to execute the Cargo that's being tested.

Auto merge of #2227 - tbu-:pr_pycache, r=alexcrichton

Running `make` creates the `src/etc/__pycache__` directory which should
not be checked in.

Add __pycache__ to .gitignore

Running `make` creates the `src/etc/__pycache__` directory which should
not be checked in.

remove unused imports

Auto merge of #2219 - matklad:encodable-audit, r=alexcrichton

@alexcrichton another preparation PR for #2196

I've removed obscure `metadata` field from `Target`. It is a breaking change (for read-manifest), but the field seemed cryptic, useless and untested :)

`Target` has a bunch of boolean fields:

```
    tested: bool,
    benched: bool,
    doc: bool,
    doctest: bool,
    harness: bool, // whether to use the test harness (--test)
    for_host: bool,
```

I guess they should not be included in serialized representation?

I will push commits for other `Encodable`s here.

add more fields to SerializedDependency

add more field to SerializedPackage

Pass link search path to doctests even if build script gave no links

It is entirely possible for a crate to have a build script that is simply
the equivalent to

```rustc
fn main() {
    println!("cargo:rustc-link-search=native=/some/path");
}
```

Without actually giving anything to link (for example, because the code
contains `#[link(name="foo")]`. In this case, we aren't actually passing
`-L` through when running doctests, even though they're passed when
compiling the main crate.

Fixes #1592

Allow specifying profiles to `cargo rustc`

This should allow compiling the specified target in the various profiles that
are available to it, e.g. bench or test in addition to the standard
dev.

Closes #2120

Consolidate creating processes in tests

Each test wants to be sure to reset HOME and remove CARGO_HOME from the
environment, but this was done inconsistently throughout the test suite. This
commit consolidates process creation so there's only one point for creating a
process ready to execute the Cargo that's being tested.

remove unused Encodable impl for Manifest

Auto merge of #2217 - alexcrichton:cleanup, r=alexcrichton

This From impl has been around for awhile.

Use try! instead of CliError::from_boxed

This From impl has been around for awhile.

refactor Target serialization

Remove obscure `metadata` field and implement proper Encodable for
TargetKind becase default one is not used.

Auto merge of #2215 - JIghtuse:master, r=alexcrichton

Fixes #2197

etc: Fix error message about missing Cargo snapshot

Fixes #2197

Auto merge of #2208 - leodasvacas:clippy, r=alexcrichton

Changes suggested by clippy. Mostly replacing `to_string()` with alternatives.

clippy run

Auto merge of #2209 - felixc:master, r=alexcrichton

Update zsh completion to include new 'install' command

Auto merge of #2207 - fhartwig:unused-import, r=alexcrichton

Remove unused import that breaks the build on rust nightly

Auto merge of #2204 - alexcrichton:bump, r=brson

r? @brson

Bump to 0.8.0

Auto merge of #2165 - psyomn:fix-2125-dup-bins, r=alexcrichton

Alex, thank you for directing me to the right way.

I did some changes, and made it bail with the following message if duplicate binaries are detected:

````nocode
failed to parse manifest at `/home/psyomn/programming/rust/fork/badproj/Cargo.toml`

Caused by:
  rejected binary name e: only unique binary names are accepted
````

I also added a test to check for this! Let me know if anything needs fixing.

ref #2125

fix #2125: duplicate binary names will cause a bail

* adds duplicate target checks for bins, tests, examples, benches
* adds tests to test the above

Auto merge of #2201 - matklad:serialized-dep, r=alexcrichton

This hides `SerializedDependency` from general public, as requested [here](https://github.com/rust-lang/cargo/pull/1434#issuecomment-97886950). It also hides `SerializedManifest` which was (wrongly?) exposed.

This is required for #2196. I want to move in small steps this time, hence the separate PR.

Technically this break backwards compatibility, because `SerializedDependency` and `SerializedManifest` were public (`SerializedPackage` was private however). Are such changes allowed in cargo?

implement Encodable for Dependency

SerializedDependency can be a private implementation detail now.

make SerializedManifest private

Auto merge of #2192 - alexcrichton:search-cargo-home, r=brson

Don't require PATH modifications for new cargo subcommands by looking
specifically in $CARGO_HOME/bin for installed commands.

Closes #2189

Probe $CARGO_HOME/bin for subcommands by default

Don't require PATH modifications for new cargo subcommands by looking
specifically in $CARGO_HOME/bin for installed commands.

Closes #2189

Auto merge of #2194 - steveklabnik:update_semver, r=alexcrichton

I realized that Cargo wasn't using the latest semver. This doesn't include my big nom refactoring, that will be 0.3.0.

Update semver to 0.2.0

Auto merge of #2134 - alexcrichton:build-script-input, r=brson

Currently Cargo is quite conservative in how it determines whether a build
script should be run. The heuristic used is "did any file in the project
directory change", but this is almost always guaranteed to be too coarse
grained in situations like:

* If the build script takes a long time to run it's advantageous to run it as
  few times as possible. Being able to inform Cargo about precisely when a build
  script should be run should provide more robust support here.
* Build scripts may not always have all of their dependencies in-tree or in the
  crate root. Sometimes a dependency could be elsewhere in a repository and
  scripts need a method of informing Cargo about this (as currently these
  compiles don't happen then they should).

This commit adds this support in build scripts via a new `rerun-if-changed`
directive which can be printed to standard output (using the standard Cargo
metadata format). The value for this key is a path relative to the crate root,
and Cargo will only look at these paths when determining whether to rerun the
build script. Any other file changes will not trigger the build script to be
rerun.

Currently the printed paths may either be a file or a directory, and a directory
is deeply traversed. The heuristic for trigger a rerun is detecting whether any
input file has been modified since the build script was last run (determined by
looking at the modification time of the output file of the build script). This
current implementation means that if you depend on a directory and then delete a
file within it the build script won't be rerun, but this is already the case and
can perhaps be patched up later.

Future extensions could possibly include the usage of glob patterns in build
script paths like the `include` and `exclude` features of `Cargo.toml`, but
these should be backwards compatible to add in the future.

Closes #1162

Auto merge of #2190 - sfackler:wildcard-pushback, r=alexcrichton

We missed a release somewhere, so the warning here won't hit stable
until December 11th.

r? @alexcrichton

Move wildcard rejection date back 6 weeks

We missed a release somewhere, so the warning here won't hit stable
until December 11th.

Auto merge of #2186 - alexcrichton:transaction, r=brson

Right now a `Bomb` struct is used to attempt to ensure that broken tarballs
don't escape, but this unfortunately doesn't work for when Cargo is terminated
via other means such as ctrl-c or abnormal termination. Instead the tarball is
constructed in a temporary location and then only moved to the final location
once all checks pass.

Closes #2173
cc #2177

Auto merge of #2185 - alexcrichton:update, r=alexcrichton

Update plugin tests to nightly

Auto merge of #2184 - brandonson:master, r=alexcrichton

When the crate to install is unspecified, we should really inform users about
where they can install from (other than crates.io).  This makes the message mention --path
and --git so that people know they exist.